Skip to content

Entity Actions: Create button discernible text (fixes #20205)#20434

Merged
leekelleher merged 18 commits intoumbraco:mainfrom
OskarKruger:CreateButtonMissingText
Oct 9, 2025
Merged

Entity Actions: Create button discernible text (fixes #20205)#20434
leekelleher merged 18 commits intoumbraco:mainfrom
OskarKruger:CreateButtonMissingText

Conversation

@OskarKruger
Copy link
Copy Markdown
Contributor

Description

Added proper localization support for entity action buttons across different sections(content and media to start with), improving accessibility. The changes ensure consistent localization behavior for both "Create" and "View Actions" buttons along with support for a screen reader, addressing #20205 .

The current setup is made with implementing direct get methods in packages/core/entity-action/entity-action.interface.ts, which seems like it might not be architecturally correct.

Added localized key for English and German lang for testing.

@leekelleher leekelleher self-requested a review October 9, 2025 10:32
as `actions_viewActionsFor` already existed.

Also made into a function, to support a fallback label.
Removed the need for a `getCreateAriaLabel()` method.
as the "umb-entity-actions-bundle" component handles this now.
# Conflicts:
#	src/Umbraco.Web.UI.Client/src/packages/core/tree/tree-item/tree-item-base/tree-item-element-base.ts
@leekelleher leekelleher changed the title Create button missing text Entity Actions: Create button discernible text (fixes #20205) Oct 9, 2025
Copy link
Copy Markdown
Member

@leekelleher leekelleher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested out, works well! 🚀

I made some code amends, to hopefully simplify the localization of the "Create item for" text.

@leekelleher leekelleher enabled auto-merge (squash) October 9, 2025 13:22
@leekelleher leekelleher merged commit a4c373d into umbraco:main Oct 9, 2025
27 checks passed
leekelleher added a commit that referenced this pull request Oct 14, 2025
 (#20458)

* added hovering and focus border to RTE

* fix main to OG

* fix to main again

* I'm going to cry

* Missing localiztion feature, maybe UmbLitElement?

* added localization controller to fetch localized version

* localization successful for viewActionsFor and CreateFor

* clean up button text

* Changed label for content header to display proper name

* clean up code

* Included button labels for media section

* clean code

* Relocated localization keys,

as `actions_viewActionsFor` already existed.

Also made into a function, to support a fallback label.

* Simplified the "Create for" label/localization

Removed the need for a `getCreateAriaLabel()` method.

* Removed the double-localizations (of `actions_viewActionsFor`)

as the "umb-entity-actions-bundle" component handles this now.

* imports tidy-up

* Simplified localization key condition

* switched to new localization key for other sections for new labeling

* Bumped `@umbraco/playwright-testhelpers` 16.0.55

https://github.com/umbraco/Umbraco.Playwright.Testhelpers/releases/tag/release%2F16.0.55

---------

Co-authored-by: Oskar kruger <obk@umbraco.dk>
Co-authored-by: leekelleher <leekelleher@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants